Skip to content

Conversation

@ashwin-ant
Copy link
Collaborator

Add cross-platform testing for Windows alongside Linux across all test jobs (unit tests, e2e tests, and examples). Uses native Windows installation via PowerShell script and platform-specific timeout handling for example scripts.

🤖 Generated with Claude Code

ashwin-ant and others added 4 commits October 7, 2025 16:20
Add cross-platform testing for Windows alongside Linux across all test jobs (unit tests, e2e tests, and examples). Uses native Windows installation via PowerShell script and platform-specific timeout handling for example scripts.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
After installing Claude Code via PowerShell script, add the installation directory to GitHub Actions PATH so the claude command is available in subsequent steps.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
The PowerShell installer places claude.exe in $USERPROFILE\.local\bin, not $LOCALAPPDATA\Programs\claude-code. Update PATH configuration to match actual installation location.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Add 0.5s delay on Windows after client disconnect to allow subprocess file handles to be released before TemporaryDirectory cleanup. Fixes PermissionError in Python 3.11+ where tempfile cleanup became more aggressive.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@ashwin-ant ashwin-ant marked this pull request as ready for review October 7, 2025 23:59
@ashwin-ant ashwin-ant requested a review from a team October 7, 2025 23:59
ant-kurt
ant-kurt previously approved these changes Oct 8, 2025
Copy link

@ant-kurt ant-kurt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Should we do Mac too?

Add macOS to test and test-e2e job matrices. macOS uses the same bash installation script as Linux for Claude Code setup.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@ashwin-ant ashwin-ant merged commit 6793e40 into main Oct 8, 2025
30 checks passed
@ashwin-ant ashwin-ant deleted the ashwin/windowsci branch October 8, 2025 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants